More appveyor improvements #1236
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Experiments show caching the GHC folder makes appveyor go 2 to 3 minutes slower in the best case, and has a massively higher chance of falling over. Appveyor recommends the cache folders be < 100Mb, which this violates. Appveyor also zips the folders at the end, which is why caching is slower.
A workaround for #1212
|
Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The first commit saves me somewhere in the 1-3 minute range under good scenarios, but occasionally appveyor goes nuts and takes over an hour, which no longer happens with the new formulation.
The second makes it possible to actually navigate the appveyor page. Note that the
--arch i386flag has disappeared. I think that's OK because stack binary i386 will default to i386. If not, perhaps there is a need for it to go in astack setupflag?Since Appveyor already has MSYS and 7zip installed, we could skip them with a suitable flag to
stack setup(or by default), which would shave another minute or so off appveyor builds. I'm quite keen for stack to become the de facto appveyor toolkit, because its much easier than any alternative.CC @snoyberg